Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DRILL-8505: Fix Docker Hub automated container image builds #2932

Merged
merged 1 commit into from
Aug 3, 2024
Merged

DRILL-8505: Fix Docker Hub automated container image builds #2932

merged 1 commit into from
Aug 3, 2024
+7 −7

Conversation

jnturton
Copy link
Contributor

@jnturton jnturton commented Aug 2, 2024

DRILL-8505: Fix Docker Hub automated container image builds

Description

  1. Use a single threaded Maven build to avoid thread-unsafe Maven plugin issues.
  2. Move the latest and master tags to OpenJDK 17. Images based on on all three of OpenJDK 8, 11 and 17 continue to be published and tagged accordingly but the "default" tags of latest and master are repointed to OpenJDK 17. This affects the master image as soon as this PR is merged, while the latest image will only change with the next Drill release.

Documentation

N/A

Testing

Manual test of the Docker Hub automated build based on the temporary drill-8505 branch in the apache/drill repo. The temporary branch are resulting container image on Docker Hub are to be cleaned up after this PR is finalised.

image

@jnturton jnturton added ci CI backport-to-stable This bug fix is applicable to the latest stable release and should be considered for inclusion there labels Aug 2, 2024
@jnturton jnturton self-assigned this Aug 2, 2024
@jnturton jnturton changed the title Single threaded Maven build. DRILL-8505: Fix Docker Hub automated container image builds Aug 2, 2024
cgivre
cgivre approved these changes Aug 2, 2024
View reviewed changes
Copy link
Contributor

@cgivre cgivre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1

@jnturton jnturton merged commit 0cdb471 into master Aug 3, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cgivre cgivre cgivre approved these changes

@vvysotskyi vvysotskyi Awaiting requested review from vvysotskyi

Assignees

@jnturton jnturton

Labels
backport-to-stable This bug fix is applicable to the latest stable release and should be considered for inclusion there ci CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jnturton @cgivre